Skip to content

Add support for new unphone boards #7109

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Aug 10, 2022
Merged

Add support for new unphone boards #7109

merged 2 commits into from
Aug 10, 2022

Conversation

garethhcoleman
Copy link
Contributor

Second time doing this - accidentally closed previous #7058 in error!

Is adding a new partition scheme acceptable?

Please advise if any changes or clarification needed.

Thanks!

Description of Change:
Adds two new pin maps for the unPhone spin 8 and spin 9 to variants/unphone8 and variants/unphone9
Updated boards.txt to add definitions for unphone7 (based on Adafruit ESP32 feather) and unphone 8 and 9 (ESP32-S3)
Added a new partition scheme for 8MB flash variant of ESP32-S3 with maximum program size (7.9MB)

Tests scenarios:
Tested on our prototype hardware with a variety of test sketches, pin assignments seem correct.

Related links:
Hardware description at unphone.net and software at https://gitlab.com/hamishcunningham/unphone

@garethhcoleman garethhcoleman changed the title tested new partition, revised max flash & ram in boards.txt Add support for new unphone boards Aug 10, 2022
@SuGlider
Copy link
Collaborator

@garethhcoleman - It seems that unphone7 is missing pins_arduino.h in Variants. Please let me know.

@SuGlider SuGlider self-assigned this Aug 10, 2022
@garethhcoleman
Copy link
Contributor Author

@SuGlider Yes, you are correct.
We based the unphone7 on the esp32 feather so closely, they share pin definitions.
Therefore, we have unphone7.build.variant=feather_esp32
This works in our testing, please advise if this is acceptable to you, happy to create a copy if required.

@SuGlider SuGlider merged commit 3e65a57 into espressif:master Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants